I think this is a good change, an explicit typing improvement with no actual functionality change. Only concern might be breaking type-checking with other modules but that doesn't fall under the version compatibility anyway so probably OK...

Well, it passes the pipeline now, but locally I still get some errors:

jh@pns-admin:~/dev/pyinfra$ mypy
Warning: Unpack is already enabled by default
pyinfra/context.py:83: error: Incompatible types in assignment (expression has type "type[local]", base class "ContextObject" defined the type as "type[container]")  [assignment]
pyinfra_cli/__main__.py:34: error: Missing positional argument "greenlet" in call to "signal"  [call-arg]
pyinfra_cli/__main__.py:37: error: Module has no attribute "signal"  [attr-defined]
Found 3 errors in 2 files (checked 145 source files)

So I'm guessing there's some different version of greenlet installed in my container ?

Well, it passes the pipeline now, but locally I still get some errors:

jh@pns-admin:~/dev/pyinfra$ mypy
Warning: Unpack is already enabled by default
pyinfra/context.py:83: error: Incompatible types in assignment (expression has type "type[local]", base class "ContextObject" defined the type as "type[container]")  [assignment]
pyinfra_cli/__main__.py:34: error: Missing positional argument "greenlet" in call to "signal"  [call-arg]
pyinfra_cli/__main__.py:37: error: Module has no attribute "signal"  [attr-defined]
Found 3 errors in 2 files (checked 145 source files)

So I'm guessing there's some different version of greenlet installed in my container ?

They look unrelated though? Possibly greenlet related, I'll pull this and poke at it a bit when I can.